Skip to content

Conversation

@captainzmc
Copy link
Member

@captainzmc captainzmc commented Jun 15, 2022

What changes were proposed in this pull request?

Put EC key should not using streaming.
In the EC write test, writing a small file (less than 4MB) was successful, but writing 100MB failed. Cause 100MB EC key should not be written using Streaming.

image

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-6867

How was this patch tested?

Existing test in acceptance-unsecure.

@captainzmc captainzmc changed the base branch from master to HDDS-4454 June 15, 2022 02:44
@captainzmc captainzmc changed the title HDDS-6867. PutKeyHandler should not use streaming to put EC key. HDDS-6867. [Ozone-Streaming] PutKeyHandler should not use streaming to put EC key. Jun 15, 2022
@captainzmc captainzmc requested a review from szetszwo June 15, 2022 09:31
@captainzmc
Copy link
Member Author

Because the EC and mergeRocksDB branches merged into the master, Streaming CI reported errors, we fix these at this PR.
Hi @szetszwo, can you help review this.

Copy link
Contributor

@szetszwo szetszwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 the change looks good.

@szetszwo szetszwo merged commit a4199f5 into apache:HDDS-4454 Jun 15, 2022
captainzmc added a commit to captainzmc/hadoop-ozone that referenced this pull request Jul 4, 2022
szetszwo pushed a commit that referenced this pull request Oct 25, 2022
…o put EC key. (#3516)

(cherry picked from commit 2227d33)
(cherry picked from commit 04ebe898435bbef40228751590cd26f1d1e2f051)
szetszwo pushed a commit that referenced this pull request Nov 7, 2022
…o put EC key. (#3516)

(cherry picked from commit 2227d33)
(cherry picked from commit 04ebe898435bbef40228751590cd26f1d1e2f051)
(cherry picked from commit 29a8c15)
szetszwo pushed a commit that referenced this pull request Dec 1, 2022
szetszwo pushed a commit that referenced this pull request Dec 16, 2022
nishitpatira pushed a commit to nishitpatira/ozone that referenced this pull request Dec 16, 2022
errose28 added a commit to errose28/ozone that referenced this pull request Dec 21, 2022
* master: (88 commits)
  HDDS-7463. SCM Pipeline scrubber never able to cleanup allocated pipeline. (apache#4093)
  HDDS-7683. EC: ReplicationManager - UnderRep maintenance handler should not request nodes if none needed (apache#4109)
  HDDS-7635. Update failure metrics when allocate block fails in preExecute. (apache#4086)
  HDDS-7565. FSO purge directory for old bucket can update quota for new bucket (apache#4021)
  HDDS-7654. EC: ReplicationManager - merge mis-rep queue into under replicated queue (apache#4099)
  HDDS-7621. Update SCM term in datanode from heartbeat without any commands (apache#4101)
  HDDS-7649. S3 multipart upload EC release space quota wrong for old version (apache#4095)
  HDDS-7399. Enable specifying external root ca (apache#4053)
  HDDS-7398. Tool to remove old certs from the scm db (apache#3972)
  HDDS-6650. S3MultipartUpload support update bucket usedNamespace. (apache#4081)
  HDDS-7605. Improve logging in Container Balancer (apache#4067)
  HDDS-7616. EC: Refactor Unhealthy Replicated Processor (apache#4063)
  HDDS-7426. Add a new acceptance test for Streaming Pipeline. (apache#4019)
  HDDS-7478. [Ozone-Streaming] NPE in when creating a file with o3fs. (apache#3949)
  HDDS-7425. Add documentation for the new Streaming Pipeline feature. (apache#3913)
  HDDS-7438. [Ozone-Streaming] Add a createStreamKey method to OzoneBucket. (apache#3914)
  HDDS-7431. [Ozone-Streaming] Disable data steam by default. (apache#3900)
  HDDS-6955. [Ozone-streaming] Add explicit stream flag in ozone shell (apache#3559)
  HDDS-6867.  [Ozone-Streaming] PutKeyHandler should not use streaming to put EC key. (apache#3516)
  HDDS-6842. [Ozone-Streaming] Reduce the number of watch requests in StreamCommitWatcher. (apache#3492)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants